Update Spock Tests to run with STRICT_TEST mode by default#11032
Update Spock Tests to run with STRICT_TEST mode by default#11032
STRICT_TEST mode by default#11032Conversation
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 59 metrics, 12 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.61.0-SNAPSHOT~64af64756a, baseline=1.61.0-SNAPSHOT~b664cc0f2a
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.054 s) : 0, 1053875
Total [baseline] (8.835 s) : 0, 8835228
Agent [candidate] (1.058 s) : 0, 1057872
Total [candidate] (8.873 s) : 0, 8873297
section iast
Agent [baseline] (1.224 s) : 0, 1223539
Total [baseline] (9.578 s) : 0, 9577752
Agent [candidate] (1.224 s) : 0, 1223749
Total [candidate] (9.553 s) : 0, 9553081
gantt
title insecure-bank - break down per module: candidate=1.61.0-SNAPSHOT~64af64756a, baseline=1.61.0-SNAPSHOT~b664cc0f2a
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.235 ms) : 0, 1235
crashtracking [candidate] (1.228 ms) : 0, 1228
BytebuddyAgent [baseline] (632.124 ms) : 0, 632124
BytebuddyAgent [candidate] (633.795 ms) : 0, 633795
AgentMeter [baseline] (29.381 ms) : 0, 29381
AgentMeter [candidate] (29.287 ms) : 0, 29287
GlobalTracer [baseline] (248.854 ms) : 0, 248854
GlobalTracer [candidate] (248.548 ms) : 0, 248548
AppSec [baseline] (31.97 ms) : 0, 31970
AppSec [candidate] (31.987 ms) : 0, 31987
Debugger [baseline] (59.071 ms) : 0, 59071
Debugger [candidate] (59.142 ms) : 0, 59142
Remote Config [baseline] (602.367 µs) : 0, 602
Remote Config [candidate] (598.592 µs) : 0, 599
Telemetry [baseline] (7.997 ms) : 0, 7997
Telemetry [candidate] (8.1 ms) : 0, 8100
Flare Poller [baseline] (6.48 ms) : 0, 6480
Flare Poller [candidate] (8.958 ms) : 0, 8958
section iast
crashtracking [baseline] (1.224 ms) : 0, 1224
crashtracking [candidate] (1.223 ms) : 0, 1223
BytebuddyAgent [baseline] (801.09 ms) : 0, 801090
BytebuddyAgent [candidate] (801.354 ms) : 0, 801354
AgentMeter [baseline] (11.379 ms) : 0, 11379
AgentMeter [candidate] (11.369 ms) : 0, 11369
GlobalTracer [baseline] (239.315 ms) : 0, 239315
GlobalTracer [candidate] (239.159 ms) : 0, 239159
AppSec [baseline] (33.58 ms) : 0, 33580
AppSec [candidate] (30.835 ms) : 0, 30835
Debugger [baseline] (55.452 ms) : 0, 55452
Debugger [candidate] (58.423 ms) : 0, 58423
Remote Config [baseline] (519.942 µs) : 0, 520
Remote Config [candidate] (523.801 µs) : 0, 524
Telemetry [baseline] (13.994 ms) : 0, 13994
Telemetry [candidate] (14.34 ms) : 0, 14340
Flare Poller [baseline] (4.762 ms) : 0, 4762
Flare Poller [candidate] (4.403 ms) : 0, 4403
IAST [baseline] (26.079 ms) : 0, 26079
IAST [candidate] (25.788 ms) : 0, 25788
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.61.0-SNAPSHOT~64af64756a, baseline=1.61.0-SNAPSHOT~b664cc0f2a
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.063 s) : 0, 1062609
Total [baseline] (11.119 s) : 0, 11119344
Agent [candidate] (1.054 s) : 0, 1054231
Total [candidate] (11.046 s) : 0, 11046307
section appsec
Agent [baseline] (1.251 s) : 0, 1251281
Total [baseline] (11.22 s) : 0, 11220023
Agent [candidate] (1.246 s) : 0, 1245501
Total [candidate] (11.159 s) : 0, 11158878
section iast
Agent [baseline] (1.242 s) : 0, 1241543
Total [baseline] (11.419 s) : 0, 11419494
Agent [candidate] (1.253 s) : 0, 1253106
Total [candidate] (11.361 s) : 0, 11361354
section profiling
Agent [baseline] (1.185 s) : 0, 1185153
Total [baseline] (11.135 s) : 0, 11135239
Agent [candidate] (1.192 s) : 0, 1191604
Total [candidate] (11.095 s) : 0, 11094726
gantt
title petclinic - break down per module: candidate=1.61.0-SNAPSHOT~64af64756a, baseline=1.61.0-SNAPSHOT~b664cc0f2a
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.237 ms) : 0, 1237
crashtracking [candidate] (1.215 ms) : 0, 1215
BytebuddyAgent [baseline] (637.346 ms) : 0, 637346
BytebuddyAgent [candidate] (631.466 ms) : 0, 631466
AgentMeter [baseline] (29.521 ms) : 0, 29521
AgentMeter [candidate] (29.384 ms) : 0, 29384
GlobalTracer [baseline] (250.169 ms) : 0, 250169
GlobalTracer [candidate] (248.776 ms) : 0, 248776
AppSec [baseline] (31.97 ms) : 0, 31970
AppSec [candidate] (32.087 ms) : 0, 32087
Debugger [baseline] (60.052 ms) : 0, 60052
Debugger [candidate] (59.942 ms) : 0, 59942
Remote Config [baseline] (596.386 µs) : 0, 596
Remote Config [candidate] (624.784 µs) : 0, 625
Telemetry [baseline] (8.068 ms) : 0, 8068
Telemetry [candidate] (8.047 ms) : 0, 8047
Flare Poller [baseline] (7.297 ms) : 0, 7297
Flare Poller [candidate] (6.63 ms) : 0, 6630
section appsec
crashtracking [baseline] (1.236 ms) : 0, 1236
crashtracking [candidate] (1.209 ms) : 0, 1209
BytebuddyAgent [baseline] (661.375 ms) : 0, 661375
BytebuddyAgent [candidate] (660.756 ms) : 0, 660756
AgentMeter [baseline] (12.115 ms) : 0, 12115
AgentMeter [candidate] (11.998 ms) : 0, 11998
GlobalTracer [baseline] (250.228 ms) : 0, 250228
GlobalTracer [candidate] (248.516 ms) : 0, 248516
IAST [baseline] (24.79 ms) : 0, 24790
IAST [candidate] (24.553 ms) : 0, 24553
AppSec [baseline] (185.528 ms) : 0, 185528
AppSec [candidate] (183.416 ms) : 0, 183416
Debugger [baseline] (66.636 ms) : 0, 66636
Debugger [candidate] (66.103 ms) : 0, 66103
Remote Config [baseline] (610.685 µs) : 0, 611
Remote Config [candidate] (612.965 µs) : 0, 613
Telemetry [baseline] (8.727 ms) : 0, 8727
Telemetry [candidate] (8.557 ms) : 0, 8557
Flare Poller [baseline] (3.6 ms) : 0, 3600
Flare Poller [candidate] (3.506 ms) : 0, 3506
section iast
crashtracking [baseline] (1.244 ms) : 0, 1244
crashtracking [candidate] (1.26 ms) : 0, 1260
BytebuddyAgent [baseline] (812.764 ms) : 0, 812764
BytebuddyAgent [candidate] (825.217 ms) : 0, 825217
AgentMeter [baseline] (11.598 ms) : 0, 11598
AgentMeter [candidate] (12.18 ms) : 0, 12180
GlobalTracer [baseline] (242.213 ms) : 0, 242213
GlobalTracer [candidate] (241.915 ms) : 0, 241915
IAST [baseline] (26.246 ms) : 0, 26246
IAST [candidate] (26.147 ms) : 0, 26147
AppSec [baseline] (30.823 ms) : 0, 30823
AppSec [candidate] (33.531 ms) : 0, 33531
Debugger [baseline] (60.804 ms) : 0, 60804
Debugger [candidate] (56.724 ms) : 0, 56724
Remote Config [baseline] (524.471 µs) : 0, 524
Remote Config [candidate] (537.91 µs) : 0, 538
Telemetry [baseline] (14.306 ms) : 0, 14306
Telemetry [candidate] (14.356 ms) : 0, 14356
Flare Poller [baseline] (4.632 ms) : 0, 4632
Flare Poller [candidate] (4.249 ms) : 0, 4249
section profiling
crashtracking [baseline] (1.184 ms) : 0, 1184
crashtracking [candidate] (1.189 ms) : 0, 1189
BytebuddyAgent [baseline] (691.708 ms) : 0, 691708
BytebuddyAgent [candidate] (697.007 ms) : 0, 697007
AgentMeter [baseline] (9.17 ms) : 0, 9170
AgentMeter [candidate] (9.127 ms) : 0, 9127
GlobalTracer [baseline] (206.953 ms) : 0, 206953
GlobalTracer [candidate] (207.936 ms) : 0, 207936
AppSec [baseline] (32.653 ms) : 0, 32653
AppSec [candidate] (32.622 ms) : 0, 32622
Debugger [baseline] (65.999 ms) : 0, 65999
Debugger [candidate] (66.229 ms) : 0, 66229
Remote Config [baseline] (575.743 µs) : 0, 576
Remote Config [candidate] (564.679 µs) : 0, 565
Telemetry [baseline] (7.924 ms) : 0, 7924
Telemetry [candidate] (7.958 ms) : 0, 7958
Flare Poller [baseline] (3.565 ms) : 0, 3565
Flare Poller [candidate] (3.635 ms) : 0, 3635
ProfilingAgent [baseline] (94.159 ms) : 0, 94159
ProfilingAgent [candidate] (93.771 ms) : 0, 93771
Profiling [baseline] (94.726 ms) : 0, 94726
Profiling [candidate] (94.338 ms) : 0, 94338
LoadParameters
See matching parameters
SummaryFound 3 performance improvements and 2 performance regressions! Performance is the same for 16 metrics, 15 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.61.0-SNAPSHOT~64af64756a, baseline=1.61.0-SNAPSHOT~b664cc0f2a
dateFormat X
axisFormat %s
section baseline
no_agent (1.23 ms) : 1218, 1241
. : milestone, 1230,
iast (3.246 ms) : 3199, 3293
. : milestone, 3246,
iast_FULL (6.143 ms) : 6080, 6206
. : milestone, 6143,
iast_GLOBAL (3.726 ms) : 3660, 3791
. : milestone, 3726,
profiling (2.555 ms) : 2529, 2581
. : milestone, 2555,
tracing (1.927 ms) : 1910, 1945
. : milestone, 1927,
section candidate
no_agent (1.25 ms) : 1237, 1262
. : milestone, 1250,
iast (3.332 ms) : 3285, 3379
. : milestone, 3332,
iast_FULL (6.038 ms) : 5977, 6100
. : milestone, 6038,
iast_GLOBAL (3.626 ms) : 3568, 3685
. : milestone, 3626,
profiling (2.268 ms) : 2248, 2289
. : milestone, 2268,
tracing (1.85 ms) : 1834, 1865
. : milestone, 1850,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.61.0-SNAPSHOT~64af64756a, baseline=1.61.0-SNAPSHOT~b664cc0f2a
dateFormat X
axisFormat %s
section baseline
no_agent (19.44 ms) : 19241, 19638
. : milestone, 19440,
appsec (18.536 ms) : 18349, 18723
. : milestone, 18536,
code_origins (18.744 ms) : 18558, 18931
. : milestone, 18744,
iast (17.754 ms) : 17577, 17930
. : milestone, 17754,
profiling (18.605 ms) : 18421, 18789
. : milestone, 18605,
tracing (18.002 ms) : 17824, 18179
. : milestone, 18002,
section candidate
no_agent (19.384 ms) : 19187, 19580
. : milestone, 19384,
appsec (18.566 ms) : 18375, 18756
. : milestone, 18566,
code_origins (17.734 ms) : 17560, 17909
. : milestone, 17734,
iast (18.653 ms) : 18466, 18839
. : milestone, 18653,
profiling (18.697 ms) : 18511, 18884
. : milestone, 18697,
tracing (17.778 ms) : 17600, 17955
. : milestone, 17778,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 10 metrics, 2 unstable metrics. Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.61.0-SNAPSHOT~64af64756a, baseline=1.61.0-SNAPSHOT~b664cc0f2a
dateFormat X
axisFormat %s
section baseline
no_agent (15.025 s) : 15025000, 15025000
. : milestone, 15025000,
appsec (14.732 s) : 14732000, 14732000
. : milestone, 14732000,
iast (18.687 s) : 18687000, 18687000
. : milestone, 18687000,
iast_GLOBAL (18.101 s) : 18101000, 18101000
. : milestone, 18101000,
profiling (15.068 s) : 15068000, 15068000
. : milestone, 15068000,
tracing (14.772 s) : 14772000, 14772000
. : milestone, 14772000,
section candidate
no_agent (15.394 s) : 15394000, 15394000
. : milestone, 15394000,
appsec (14.795 s) : 14795000, 14795000
. : milestone, 14795000,
iast (18.162 s) : 18162000, 18162000
. : milestone, 18162000,
iast_GLOBAL (17.82 s) : 17820000, 17820000
. : milestone, 17820000,
profiling (15.319 s) : 15319000, 15319000
. : milestone, 15319000,
tracing (14.693 s) : 14693000, 14693000
. : milestone, 14693000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.61.0-SNAPSHOT~64af64756a, baseline=1.61.0-SNAPSHOT~b664cc0f2a
dateFormat X
axisFormat %s
section baseline
no_agent (1.487 ms) : 1476, 1499
. : milestone, 1487,
appsec (3.744 ms) : 3527, 3962
. : milestone, 3744,
iast (2.269 ms) : 2200, 2338
. : milestone, 2269,
iast_GLOBAL (2.313 ms) : 2244, 2382
. : milestone, 2313,
profiling (2.528 ms) : 2356, 2701
. : milestone, 2528,
tracing (2.082 ms) : 2028, 2135
. : milestone, 2082,
section candidate
no_agent (1.498 ms) : 1486, 1509
. : milestone, 1498,
appsec (3.841 ms) : 3618, 4064
. : milestone, 3841,
iast (2.278 ms) : 2209, 2347
. : milestone, 2278,
iast_GLOBAL (2.314 ms) : 2245, 2383
. : milestone, 2314,
profiling (2.121 ms) : 2065, 2178
. : milestone, 2121,
tracing (2.07 ms) : 2017, 2123
. : milestone, 2070,
|
5ba6f69 to
b0147c9
Compare
95b7e3f to
82a89e0
Compare
|
@codex review |
|
ok so most of the changes are coming from the test files where we override that global config |
|
Codex Review: Didn't find any major issues. What shall we delve into next? ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in Use
⏳ Processing |
What Does This Do
This PR updates Spock tests to run with
STRICT_TESTby default. This means that all Environment Variables that are queried but not documented will fail the test. We fail the test by throwing an error when querying such environment variable, but if the test has automatic error handling we also throw an error during test cleanup to log the missing configurations fromsupported-configurations.json.If a test is using fake configurations, the test itself can override the strictness mode with
TESTto bypass this mechanism. Note that this is being done w/ many Instrumentation tests as they create fake decorators that register fake configs.Motivation
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]
Note: Once your PR is ready to merge, add it to the merge queue by commenting
/merge./merge -ccancels the queue request./merge -f --reason "reason"skips all merge queue checks; please use this judiciously, as some checks do not run at the PR-level. For more information, see this doc.